From: Florian Müllner Date: Tue, 12 Feb 2019 12:37:39 +0000 (+0100) Subject: appIconMenu: Sync windows section with app menu X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~845^2^2~271 X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=56a361650c829e3026c80eeb04236cedd55e7a98;p=gnome-shell.git appIconMenu: Sync windows section with app menu Keep the app icon context menu consistent with the app menu, so stop grouping windows by workspace and add a section heading. https://gitlab.gnome.org/GNOME/gnome-shell/issues/968 --- diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js index adaefa7dd..16df9ca0d 100644 --- a/js/ui/appDisplay.js +++ b/js/ui/appDisplay.js @@ -1707,25 +1707,20 @@ var AppIconMenu = class AppIconMenu extends PopupMenu.PopupMenu { w => !w.skip_taskbar ); - // Display the app windows menu items and the separator between windows - // of the current desktop and other windows. - let workspaceManager = global.workspace_manager; - let activeWorkspace = workspaceManager.get_active_workspace(); - let separatorShown = windows.length > 0 && windows[0].get_workspace() != activeWorkspace; - - for (let i = 0; i < windows.length; i++) { - let window = windows[i]; - if (!separatorShown && window.get_workspace() != activeWorkspace) { - this._appendSeparator(); - separatorShown = true; - } + if (windows.length > 0) + this.addMenuItem( + /* Translators: This is the heading of a list of open windows */ + new PopupMenu.PopupSeparatorMenuItem(_("Open Windows")) + ); + + windows.forEach(window => { let title = window.title ? window.title : this._source.app.get_name(); let item = this._appendMenuItem(title); item.connect('activate', () => { this.emit('activate-window', window); }); - } + }); if (!this._source.app.is_window_backed()) { this._appendSeparator();